Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP] User Info #2474

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from
Draft

[WIP] User Info #2474

wants to merge 23 commits into from

Conversation

G-Ambatte
Copy link
Collaborator

@G-Ambatte G-Ambatte commented Oct 28, 2022 •
edited
Loading

This draft PR is to demonstrate the progress of adding a new UserInfo collection to the current NaturalCrit database used by Homebrewery.

This PR will resolve #1908 when complete.

Current goals:

  • add last user activity to UserInfo
    • create user with default properties if not already present
  • add ability to look up UserInfo to Admin page
    • fix Admin page
  • add user options to global object
  • move Recent Brews data storage to UserInfo
  • user awards?
  • ???

@G-Ambatte G-Ambatte self-assigned this Oct 28, 2022
@G-Ambatte G-Ambatte mentioned this pull request Feb 14, 2023
Open
6 tasks
@calculuschild calculuschild temporarily deployed to homebrewery-pr-2474 April 24, 2023 18:53 Inactive
@calculuschild calculuschild temporarily deployed to homebrewery-pr-2474 August 29, 2023 06:02 Inactive
@calculuschild calculuschild temporarily deployed to homebrewery-pr-2474 August 29, 2023 06:23 Inactive
@calculuschild calculuschild temporarily deployed to homebrewery-pr-2474 January 17, 2024 17:23 Inactive
@calculuschild calculuschild temporarily deployed to homebrewery-pr-2474 January 18, 2024 20:05 Inactive
@dbolack-ab
Copy link
Collaborator

Looking at this, I am curious about the unchecked boxes. Can we expand those to show what information is going in there?

@dbolack-ab
Copy link
Collaborator

I think a "Display Name" would be nice as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calculuschild calculuschild Awaiting requested review from calculuschild

At least 2 approving reviews are required to merge this pull request.

Assignees

@G-Ambatte G-Ambatte

Labels
None yet
Projects
@calculuschild's backlog
Status: Backlog
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

User settings
3 participants
@G-Ambatte @dbolack-ab @calculuschild